Skip to content

fix: use /usr/bin/env shebangs for portability across systems#753

Open
sveitser wants to merge 1 commit intortk-ai:developfrom
sveitser:fix-shebangs
Open

fix: use /usr/bin/env shebangs for portability across systems#753
sveitser wants to merge 1 commit intortk-ai:developfrom
sveitser:fix-shebangs

Conversation

@sveitser
Copy link

Summary

  • Currently the scripts don't run on NixOS
  • /usr/bin/env bash is more portable

Test plan

It's a tiny change and easy to inspect visually.

@CLAassistant
Copy link

CLAassistant commented Mar 20, 2026

CLA assistant check
All committers have signed the CLA.

@sveitser
Copy link
Author

@CLAassistant check

@aeppling
Copy link
Contributor

Hey

We are cleaning up the codebase and improving the project structure for better onboarding. As part of this effort, PR #826 reorganizes src/ from a flat layout into subfolders.

No logic changes — only file moves and import path updates.

What you need to do

Rebase your branch on develop when receiving this comment:

git fetch origin && git rebase origin/develop

Git detects renames automatically. If you get import conflicts, update the paths:

use crate::git;        // now: use crate::cmds::git::git;
use crate::tracking;   // now: use crate::core::tracking;
use crate::config;     // now: use crate::core::config;
use crate::init;       // now: use crate::hooks::init;
use crate::gain;       // now: use crate::analytics::gain;

Need help rebasing? Tag @aeppling

@sveitser
Copy link
Author

@aeppling rebased, could you review?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants